Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[improve][ci] Detect test thread leaks in CI builds and add tooling for resource leak investigation #153

Closed
wants to merge 6 commits into from

Conversation

lhotari
Copy link
Owner

@lhotari lhotari commented Oct 17, 2023

No description provided.

@codecov
Copy link

codecov bot commented Oct 17, 2023

Codecov Report

Merging #153 (9b371f4) into master (f51e7ef) will increase coverage by 0.44%.
Report is 20 commits behind head on master.
The diff coverage is 76.26%.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #153      +/-   ##
============================================
+ Coverage     72.80%   73.25%   +0.44%     
- Complexity    32381    32580     +199     
============================================
  Files          1888     1890       +2     
  Lines        140298   140357      +59     
  Branches      15420    15425       +5     
============================================
+ Hits         102144   102818     +674     
+ Misses        30063    29446     -617     
- Partials       8091     8093       +2     
Flag Coverage Δ
inttests 24.21% <38.88%> (?)
systests 24.72% <34.34%> (+0.05%) ⬆️
unittests 72.54% <76.26%> (+0.35%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
...authentication/AuthenticationDataSubscription.java 64.28% <100.00%> (+2.74%) ⬆️
...rization/MultiRolesTokenAuthorizationProvider.java 60.19% <100.00%> (+1.01%) ⬆️
...ache/pulsar/broker/resources/ClusterResources.java 73.80% <100.00%> (+3.14%) ⬆️
.../pulsar/broker/resources/LoadBalanceResources.java 100.00% <100.00%> (ø)
...pache/pulsar/broker/resources/PulsarResources.java 84.78% <100.00%> (ø)
...n/java/org/apache/pulsar/broker/PulsarService.java 82.81% <ø> (+0.23%) ⬆️
...pache/pulsar/broker/admin/impl/NamespacesBase.java 72.92% <100.00%> (+0.84%) ⬆️
.../org/apache/pulsar/broker/admin/v2/Namespaces.java 68.20% <ø> (-0.19%) ⬇️
.../pulsar/broker/service/AbstractBaseDispatcher.java 94.32% <100.00%> (+1.10%) ⬆️
...ava/org/apache/pulsar/broker/service/Consumer.java 86.23% <ø> (ø)
... and 34 more

... and 115 files with indirect coverage changes

@lhotari lhotari force-pushed the lh-trace-test-resources-cleanup branch from ac9ddfe to ec2fcad Compare October 18, 2023 10:31
@github-actions github-actions bot added the PIP label Oct 18, 2023
@lhotari lhotari changed the title [improve][ci] Add way to collect threaddump and heap histogram from test execution [improve][ci] Detect test thread leaks in CI builds and add tooling for resource leak investigation Oct 18, 2023
@lhotari lhotari force-pushed the lh-trace-test-resources-cleanup branch from ec2fcad to 3162692 Compare October 18, 2023 11:36
@github-actions github-actions bot removed the PIP label Oct 18, 2023
@lhotari lhotari force-pushed the lh-trace-test-resources-cleanup branch 2 times, most recently from c71ca4f to 5e480c5 Compare October 20, 2023 12:53
@lhotari lhotari force-pushed the master branch 7 times, most recently from 9993785 to f51e7ef Compare October 21, 2023 04:56
@lhotari lhotari force-pushed the lh-trace-test-resources-cleanup branch 2 times, most recently from 59f3f09 to 252316a Compare October 23, 2023 15:18
@lhotari lhotari closed this Oct 23, 2023
@lhotari lhotari reopened this Oct 23, 2023
@lhotari lhotari force-pushed the lh-trace-test-resources-cleanup branch from 0a52956 to 18ca4cc Compare October 24, 2023 06:48
@github-actions github-actions bot added the PIP label Oct 24, 2023
@lhotari lhotari force-pushed the lh-trace-test-resources-cleanup branch from 18ca4cc to 0e8928f Compare October 24, 2023 13:01
@github-actions github-actions bot removed the PIP label Oct 24, 2023
@lhotari lhotari force-pushed the lh-trace-test-resources-cleanup branch from 0e8928f to ea16838 Compare October 25, 2023 14:17
@lhotari lhotari force-pushed the lh-trace-test-resources-cleanup branch from ea16838 to 9b371f4 Compare October 26, 2023 14:57
@lhotari lhotari closed this Oct 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants